Skip to content

refactor: standardize Mempool glass/Goggles category naming to 'spam'#12

Open
twofaktor wants to merge 1 commit into
Retropex:masterfrom
twofaktor:data_to_spam_fix
Open

refactor: standardize Mempool glass/Goggles category naming to 'spam'#12
twofaktor wants to merge 1 commit into
Retropex:masterfrom
twofaktor:data_to_spam_fix

Conversation

@twofaktor

@twofaktor twofaktor commented Jul 3, 2026

Copy link
Copy Markdown

Summary

This PR standardizes the naming of the arbitrary data embedding category across both Mempool Glass and Mempool Goggles.

The two components were using different identifiers for the same category: Mempool Glass uses spam, while parts of the Mempool Goggles implementation still referenced it as data. This inconsistency made the codebase harder to follow and maintain.

This change aligns Mempool Goggles with the existing Mempool Glass naming by consistently using spam throughout both implementations.

imagenimagen

Changes

  • Renamed all remaining data category references in Mempool Goggles to spam.
  • Updated the affected mappings and related code paths to use the same identifier as Mempool Glass.
  • No functional changes to the filters themselves; this is purely a naming consistency improvement.

Result

Mempool Glass and Mempool Goggles now use the same category identifier (spam) for arbitrary data embedding filters, improving consistency across the project, making the codebase easier to maintain, and reducing the risk of future inconsistencies.

@twofaktor twofaktor changed the title refactor: standardize Mempool Goggles 'data' category to 'spam' refactor: standardize Mempool glass/Goggles 'data' category to 'spam' Jul 3, 2026
@twofaktor twofaktor changed the title refactor: standardize Mempool glass/Goggles 'data' category to 'spam' refactor: standardize Mempool glass/Goggles category naming to 'spam' Jul 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant